Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more time for next meteor event #1493

Merged
merged 2 commits into from
Aug 4, 2024

Conversation

Ady4ik
Copy link

@Ady4ik Ady4ik commented Jul 30, 2024

Описание PR

Увеличение времени между ивентами спавна метеора. Требуется тестирование.

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 Ady4

  • tweak: Увеличено время между роем метеоров.

Copy link

@DexlerXD DexlerXD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

предварительно

@OwlMe1steR
Copy link

Итоги теста

Просидя 2 смены АФК.
Средний промежуток между ивентами составляет 5-10 минут.
image

Промежуток между ивентами с метеорами составляет час.
image

@Kirus59
Copy link
Collaborator

Kirus59 commented Aug 1, 2024

@OwlMe1steR есть еще ивент с космической пылью (GameRuleSpaceDustMinor и GameRuleSpaceDustMajor), которая также наносит ущерб станции, хоть и меньше чем метеоры.
И вызываются они одним ивентом, с шансом 2/3 будет пыль, а с шансом 1/3 будут меторы.

Также не совсем понимаю смысла от подсчета среднего времени между ивентами, когда это итак понятно из кода ((20 + 24)/2 = 22 минуты в среднем, а учитывая что шанс на метеоры 1/3 => и получаем в среднем метеоры раз в час).

Я считаю что лучше затестить общий ущерб станции, нанесенный за 2 часа (+/- средняя продолжительность смены) метеорами и пылью, чтобы понять насколько опасен этот ивент и сколько работы предстоит инженерам выполнить за смену.

Copy link

@OwlMe1steR OwlMe1steR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Итоги теста

Провёл тестирование метеоритов на карте Frankenstein, считаю, что на всех остальных картах будут те же самые результаты.
Время указано с начала смены.

23:53 Малые метеориты
Побег сингуляронсти.
Уничтожение комнаты с ДАМом.
Частично сломана ИИшка.

47:17 Огромные метеориты
Не критичное повреждения отбытия + техи РнД.
Критическое повреждения церкви и техи над церквью.
Критическое повреждения северных соляр (для починки нужны платы и соляры).

1:06:53 Малые метеориты
Не критическо повреждение техов СБ.
Уничтожение шаттла СБ.
Уничтожение оружейки СБ.
Уничтожение каюту ГСБ.

1:30:06 Малые метеориты
Не обнаружил, что было сломано/повреждено/уничтожено, так как предыдущие события уже нанесли значительный урон.
После данного события прекратил тесты из-за их неэффективности и незначительности.

@Ady4ik решай надо ли менять время

@TheArturZh TheArturZh merged commit 02e5ae4 into SerbiaStrong-220:master Aug 4, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Увеличить время между спавнами метеоритов
5 participants